Skip to content

Conversation

@ShubhamDesai
Copy link
Contributor

This pull request fixes issue identified by Coverity Scan (CID : 1208276)

@github-actions github-actions bot added C Related code is in C module imagery labels Oct 7, 2025
ShubhamDesai and others added 4 commits October 7, 2025 13:19
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@nilason nilason added this to the 8.5.0 milestone Oct 7, 2025
@github-project-automation github-project-automation bot moved this to In Progress in GRASS Security Oct 7, 2025
Copy link
Contributor

@nilason nilason left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, there is a similar issue CID1208244, please address that here too.

@ShubhamDesai
Copy link
Contributor Author

Oh, there is a similar issue CID1208244, please address that here too.

this issue is shown under /vector/v.rectify/target.c

or are you talking about the strcat at line 292

@nilason
Copy link
Contributor

nilason commented Oct 7, 2025

If you filter with /imagery/i.ortho.photo/i.ortho.rectify/* you should see which one I mean.

@nilason
Copy link
Contributor

nilason commented Oct 7, 2025

(Coverity scan is confused by similarly named files located in different places).

@nilason
Copy link
Contributor

nilason commented Oct 7, 2025

/imagery/i.ortho.photo/i.ortho.rectify/target.c

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@ShubhamDesai
Copy link
Contributor Author

Oh, there is a similar issue CID1208244, please address that here too.

Done

@nilason nilason merged commit da1fd60 into OSGeo:main Oct 20, 2025
27 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in GRASS Security Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C Related code is in C imagery module

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants